feat(util-dynamodb): add option for using native array over sets #6591
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
By default when an dynamo object is unmarshalled, sets are converted to native javascript sets.
This is fine for some use cases, however for other cases that rely on
JSON.stringify
whether its for an API call or mapping somewhere else sets are converted toundefined
in this process.This change gives the ability to override the usage of sets and prefer to use native arrays. I am not sure if this name is perfect but also up for anything here.
Syntax
Testing
Unit tests updated to account for use case.
Additional context
Some alternatives that I have used in the past for this are to manually clone deep with the modification like below.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.